-
Notifications
You must be signed in to change notification settings - Fork 685
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add upload duration and sourcemap size to metrics #6194
chore: add upload duration and sourcemap size to metrics #6194
Conversation
🦋 Changeset detectedLatest commit: 1e23c71 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
A wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/9940208818/npm-package-wrangler-6194 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/6194/npm-package-wrangler-6194 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/9940208818/npm-package-wrangler-6194 dev path/to/script.js Additional artifacts:npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/9940208818/npm-package-create-cloudflare-6194 --no-auto-update npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/9940208818/npm-package-cloudflare-kv-asset-handler-6194 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/9940208818/npm-package-miniflare-6194 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/9940208818/npm-package-cloudflare-pages-shared-6194 npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/9940208818/npm-package-cloudflare-vitest-pool-workers-6194 Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
35050fc
to
9c48ac3
Compare
9c48ac3
to
01553f5
Compare
8673de3
to
154ba27
Compare
Adds `durationMs` and `sourceMapSize` properties to the `deploy worker script` metrics event as well as sending the event after the deploy request.
154ba27
to
1e23c71
Compare
What this PR solves / how to test
Adds
durationMs
andsourceMapSize
properties to thedeploy worker script
metrics event as well as sending the event after the deploy request. This PR can be tested by deploying a Worker using wrangler built from this PR and checking internal tools.Fixes WP-996
Author has addressed the following